Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Beyond foundation #751

Merged
merged 28 commits into from
Sep 12, 2023
Merged

Conversation

EgbertRijke
Copy link
Collaborator

In this PR I pulled changes I made in the foundation and foundation-core folders in the Beyond finite sets PR #623. The goal is to create something mergeable without having to wait for the other unfinished changes in #623. Let's see what monstrosity I created.

@fredrik-bakke fredrik-bakke marked this pull request as draft September 11, 2023 12:01
@EgbertRijke EgbertRijke marked this pull request as ready for review September 11, 2023 13:42
@EgbertRijke
Copy link
Collaborator Author

Ready for review

@fredrik-bakke
Copy link
Collaborator

Can we have #706 merged before this one?

@EgbertRijke
Copy link
Collaborator Author

Not sure how this one didn't have any merge conflicts after your big refactor

@EgbertRijke EgbertRijke marked this pull request as draft September 11, 2023 17:25
@EgbertRijke EgbertRijke marked this pull request as ready for review September 11, 2023 19:08
@EgbertRijke
Copy link
Collaborator Author

I made all the changes I wanted to make to integrate this well with #706. This one is now ready for review.

@EgbertRijke
Copy link
Collaborator Author

Oh wait I want to make some more changes:)

@EgbertRijke
Copy link
Collaborator Author

Now I think I got them all

@fredrik-bakke
Copy link
Collaborator

Great! I will review it this evening :)

Copy link
Collaborator

@fredrik-bakke fredrik-bakke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks very good! Thanks again for cleaning up after me.

src/foundation/equivalences.lagda.md Outdated Show resolved Hide resolved
src/foundation/symmetric-binary-relations.lagda.md Outdated Show resolved Hide resolved
src/foundation/identity-systems.lagda.md Show resolved Hide resolved
src/foundation/identity-systems.lagda.md Show resolved Hide resolved
src/foundation/unordered-pairs.lagda.md Show resolved Hide resolved
@fredrik-bakke
Copy link
Collaborator

One other minor remark is, for completeness, we could also repeat all the computations for action on equivalences of type families over subuniverses, and in the process also create a file for action on equivalences of functions on subuniverses.

@EgbertRijke
Copy link
Collaborator Author

Thanks for the review! I will work on it tonight!

@EgbertRijke
Copy link
Collaborator Author

I addressed all your comments. Now it's a question of how much laws you want me to prove for the new actions on equivalences, and whether we want those laws that serve no immediate purpose to hold up the useful refactorings in this PR.

@fredrik-bakke
Copy link
Collaborator

None is fine. It was only a remark

@fredrik-bakke fredrik-bakke merged commit 58eb0fc into UniMath:master Sep 12, 2023
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants